Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[node_backup] Add support for other s3 providers #90

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

blakebyrnes
Copy link
Contributor

You can configure another s3 provider by supplying the node_backup_s3_provider, node_backup_s3_access_key_id, node_backup_s3_secret_access_key, node_backup_s3_endpoint and optionally node_backup_s3_region. To use this type of service, you have to set the backup type to s3-rclone.

This change defaults to the previous configs (node_backup_r2_access_key_id, etc and a Cloudflare s3 provider.

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Dec 19, 2024

User @blakebyrnes, please sign the CLA here.

Copy link
Contributor

@nprt nprt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @blakebyrnes, thanks again for taking the time to submit this PR!

I left a couple of comments. I'd love to hear your thoughts.

@nprt
Copy link
Contributor

nprt commented Jan 2, 2025

Hey @blakebyrnes, thanks again for taking the time to submit this PR!

I left a couple of comments. I'd love to hear your thoughts.

One last thing, could you, please, also change the version here: https://github.com/paritytech/ansible-polkadot/blob/main/galaxy.yml#L11? Once we bump the version there, it should publish the new release on Ansible Galaxy.

We'll look into automating this in the future.

You can configure another s3 provider by supplying the node_backup_s3_provider, node_backup_s3_access_key_id, node_backup_s3_secret_access_key, node_backup_s3_endpoint and optionally node_backup_s3_region. To use this type of service, you have to set the backup type to s3-rclone.
Restrict the no_check_bucket to Cloudflare as the impact on other sites is unknown.

Also bumps galaxy.yml
Copy link
Contributor

@nprt nprt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, lgtm! Thank you so much for taking time to create the PR!

If you could share a bit more on how you use this, and what else could help you in your day-to-day, or in general, I'd much appreciate it.

@BulatSaif BulatSaif merged commit 47a1c88 into paritytech:main Jan 10, 2025
1 check passed
@BulatSaif
Copy link
Contributor

Thank you for contribution 👍

@BulatSaif BulatSaif changed the title Add support for other s3 providers [node_backup] Add support for other s3 providers Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants